Playwright fix test failures and improve the pytest_runtest_makereport hook

[emilghittasv]

This pull request includes several changes to improve error handling, and debugging capabilities.

The most important changes include handling the 502 errors for the test_loginless_mozilla_account_aaq test (which was the main flakiness reason for that test), extending timeouts for two messaging system tests (Sometimes SuMo returns the expected search results with a >10 seconds delay so I've bumped the wait to 15) & skipped the AAQ quote test for now .

Error Handling Improvements:

• playwright_tests/tests/ask_a_question_tests/aaq_tests/test_aaq_form_page.py: Added a check to skip the sign-in button click if a 502 error occurs and the auth page is already displayed due to a forced page refresh.
• playwright_tests/tests/conftest.py: Refactored the pytest_runtest_makereport hook to ensure video recordings are properly saved and attached to the Allure report when a test fails by taking into account not only the call.excinfo (exception is raised) but also the pytest-check failures.

Test Reliability Enhancements:

• playwright_tests/tests/ask_a_question_tests/aaq_tests/test_posted_questions.py: Skipped the test_quote_reply_functionality test until a decision is made on whether to revert a related change.
• playwright_tests/tests/explore_help_articles_tests/explore_by_topic_tests/test_explore_by_topics.py: Added navigation expectation and refresh logic to handle cases where navigation does not occur => causes the test to fail.

Timeout Adjustments:

• playwright_tests/tests/messaging_system_tests/test_messaging_system.py: Increased the timeout for verifying that no users are returned inside the messaging system search results in two separate tests. [1] [2]

[Copilot]

Pull Request Overview

This PR aims to improve error handling and test reliability in the Playwright tests. Key changes include extending timeouts in messaging system tests, handling navigation failures in explore help articles tests by implementing a refresh, and enhancing the pytest hook to properly capture video attachments when tests fail.

Reviewed Changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated no comments.

Show a summary per file

File	Description
playwright_tests/tests/messaging_system_tests/test_messaging_system.py	Increased timeout values to accommodate delayed responses during messaging system searches.
playwright_tests/tests/explore_help_articles_tests/explore_by_topic_tests/test_explore_by_topics.py	Added navigation expectation with a refresh fallback when navigation does not occur.
playwright_tests/tests/conftest.py	Refactored pytest_runtest_makereport hook for improved video saving and attaching on test failures.
playwright_tests/tests/ask_a_question_tests/aaq_tests/test_posted_questions.py	Skipped the quote reply test pending further decision.
playwright_tests/tests/ask_a_question_tests/aaq_tests/test_aaq_form_page.py	Added a conditional check to avoid clicking the sign-in button when the auth page is already active.

Comments suppressed due to low confidence (1)

playwright_tests/tests/explore_help_articles_tests/explore_by_topic_tests/test_explore_by_topics.py:43

• [nitpick] Consider replacing the print statement with a logging mechanism or an Allure step to ensure consistent and integrated reporting of navigation failures.

print("Navigation did not occur. Refreshing the page.")